Skip to content

Allow to choose the root FS location#2

Open
lcotonea wants to merge 1 commit intojammsen:mainfrom
lcotonea:main
Open

Allow to choose the root FS location#2
lcotonea wants to merge 1 commit intojammsen:mainfrom
lcotonea:main

Conversation

@lcotonea
Copy link

On my setup, I have no 'local-zfs' because my choice was to have 'local-lvm'. This simple change allow users to choose the right root fs location.

@lcotonea lcotonea changed the title Allow to choice the root FS location Allow to choose the root FS location Jan 1, 2026
@jammsen
Copy link
Owner

jammsen commented Jan 6, 2026

Can test this on the weekend, not to setup another instance with your settings first.
Can please tell me your current settings for disk-layout for a fresh proxmox9 install please?
@lcotonea

@lcotonea
Copy link
Author

lcotonea commented Jan 6, 2026

Here the disk-layout:
image
Is it enough for your setup?

@jammsen
Copy link
Owner

jammsen commented Jan 11, 2026

Here the disk-layout: image Is it enough for your setup?

I think so, sadly i cant test it right now, i need to make disk to disk clones for backups and Acronis said 10 hours this morning 😆 we are 9 hours in and it says it needs still 8-9 hours left 🤣

Im doing this as soon as i get my disks cloned and a free one tested with the hardware thats currently running Acronis. Sorry for not sticking to my offer.

@jammsen
Copy link
Owner

jammsen commented Mar 15, 2026

Finally i got my broken hardware replaced and have more test hardware for testing this.

@lcotonea - Got a handfull of questions to understand what you think about this change:

  • So what you purposing is that local-zfs is the default but the user can "manually" insert the primary storage pool themselfs right?
  • Do we need to syntax check this?
  • What coverage would you like to see? ZFS, LVM, LVM-Thin?
  • Do we need to limit it in kind of a ENUM format to make sure it doesnt break things?
  • Or does it need to be free-text for typo-possiblities?
  • How about to get from Proxmox the default via api/command?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants